Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Fixed e2e tests after removing AlertMethod.Status #602

Merged
merged 2 commits into from
Dec 4, 2024

Conversation

nobl9-adam-szymanski
Copy link
Contributor

Motivation

Error when running e2e tests:

# github.com/nobl9/nobl9-go/tests [github.com/nobl9/nobl9-go/tests.test]
Error: tests/v1alpha_alertmethod_test.go:111:9: actual.Status undefined (type alertmethod.AlertMethod has no field or method Status)

Summary

Updated tests to don't use Status field, recently removed.

Related changes

@nobl9-adam-szymanski nobl9-adam-szymanski merged commit e3f91e3 into main Dec 4, 2024
5 checks passed
@nobl9-adam-szymanski nobl9-adam-szymanski deleted the fix-tests branch December 4, 2024 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working go patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants